Skip to content

[python] Support delete_snapshot/snapshot_exists in SnapshotManager#7605

Closed
xuzifu666 wants to merge 2 commits intoapache:masterfrom
xuzifu666:snapshot_manager_opt_py
Closed

[python] Support delete_snapshot/snapshot_exists in SnapshotManager#7605
xuzifu666 wants to merge 2 commits intoapache:masterfrom
xuzifu666:snapshot_manager_opt_py

Conversation

@xuzifu666
Copy link
Copy Markdown
Member

Purpose

Currently SnapshotManager in Python does not have a method for deleting snapshots. Based on the core module, add two methods, snapshot_exists and delete_snapshot, to facilitate user use.

Tests

@JingsongLi
Copy link
Copy Markdown
Contributor

Can you explain what application scenarios?

@xuzifu666
Copy link
Copy Markdown
Member Author

xuzifu666 commented Apr 13, 2026

Can you explain what application scenarios?

I checked and found that the current Python-side snapshot manager does not yet have the function to manage and delete snapshots. Referring to the fact that the current Java-side rollback requires the ability to delete snapshots, I add it to the Python SnapshotManager.
Currently, the Python side directly deletes the snapshot directory by fileIO, but it is uncertain whether this will be needed in the future. @JingsongLi

@xuzifu666 xuzifu666 closed this Apr 13, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants